fix(docs): add FastHTTP Postgres sample run and test instructions #766
Conversation
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
|
@Achanandhi-M sir, please take a look whenever possible. |
Achanandhi-M
left a comment
Achanandhi-M
left a comment
There was a problem hiding this comment.
Hey @Srinu346, please take a look at the other quickstarts to understand how they’re structured. What you’ve added currently feels more like a README, whereas the docs follow a different format. Take your time to review how we’ve added other quickstarts and align this with our existing guides.
Also, please include a video demonstrating your changes and add relevant images as well. Make sure the images have a white background.
|
@Achanandhi-M sir i will refine it and update it. |
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
Screen.Recording.2026-01-30.001941.mp4@Achanandhi-M sir, please take a look when possible. |
|
Hi Sir, For reference, these are the images being used:
Please let me know if this is okay, or if you would prefer me to update/replace them with new screenshots. |
Achanandhi-M
left a comment
Achanandhi-M
left a comment
There was a problem hiding this comment.
Hey @Srinu346, please avoid removing the existing image from the file—don’t change anything there and use it as is. You can include additional images in the directory if needed; that’s not an issue.
Also, please include a short video showcasing your changes. One more request: kindly avoid using AI-generated text, as it appears that may have been used, and please remove emojis as well. Thanks for understanding.
2 participants
What has changed?
This PR updates the FastHTTP + Postgres sample documentation to expose and document the existing Docker-based setup that was already implemented in the backend.
The changes include:
.mdfile to document both:This PR resolves #3269
Type of change
How Has This Been Tested?
Checklist: